-
Notifications
You must be signed in to change notification settings - Fork 455
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Beginning of SQL. #163
Beginning of SQL. #163
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for a starting point!
@@ -0,0 +1,86 @@ | |||
/* | |||
* Copyright 2016 DiffPlug |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...but we should probably change this to "2017". :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2018 is coming soon enough, I was gonna change our copyright notice then :) Also want to change the notice to point to https://github.com/diffplug/spotless/graphs/contributors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds very sensible to me. :)
return input -> { | ||
return (String) formatMethod.invoke(formatter, input); | ||
}; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know if this will work with Java 9.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JLLeitschuh Are you referring to the use of reflection here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the use of the classloader in this way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm probably missing something, but I'm not yet convinced it would be a problem on Java 9, as I have a real-world project using Java 9 and Spotless with google-java-format
, and I've had no problems with Spotless or google-java-format specifically yet. (But it's worth noting that the project's using the classpath rather than the module-path, so I dunno how the JPMS would factor into things.)
#166 has this topic well-covered. |
Skeleton implementation of #156.
HibernateStep is bad - not useful enough to ship. We should delete it, but shows a good starting point.